Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

comments: Update signature and settings. #1487

Merged
merged 10 commits into from
Aug 17, 2021

Conversation

amass01
Copy link
Member

@amass01 amass01 commented Aug 11, 2021

This diff adds ExtraData & ExtraDataHint fields to the Signature
field in both New & Edit comments plugin commands.

Also, this updates the Signature validation and adds a new
comments plugin setting allowExtraData with a new validation to ensure
no extra data is provided while the setting is switched off.


Closes #1483.

- add the plugin setting with plugin setting validation
@amass01 amass01 changed the title [wip] comments: Update signature and settings comments: Update signature and settings Aug 11, 2021
@amass01 amass01 marked this pull request as ready for review August 11, 2021 16:10
Copy link
Member

@lukebp lukebp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

@amass01
Copy link
Member Author

amass01 commented Aug 16, 2021

@marcopeereboom updated

Copy link
Member

@lukebp lukebp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

@lukebp lukebp merged commit 5bc5b00 into decred:master Aug 17, 2021
@lukebp lukebp changed the title comments: Update signature and settings comments: Update signature and settings. Aug 17, 2021
@amass01 amass01 deleted the commentssignature branch August 17, 2021 13:03
@amass01 amass01 mentioned this pull request Sep 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

comments: Update signature and settings.
3 participants